Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[testnets] remove XCM SafeCallFilter for chains using Weights::v2 #4199

Merged
merged 4 commits into from
Apr 22, 2024

Conversation

acatangiu
Copy link
Contributor

@acatangiu acatangiu commented Apr 18, 2024

Weights::v2 also accounts for PoV weight so we no longer need the SafeCallFilter. All calls are allowed as long as they "fit in the block".

Weights::v3 also accounts for PoV weight so we no longer need the SafeCallFilter.
All calls are allowed as long as they "fit in the block".
@acatangiu acatangiu added the T6-XCM This PR/Issue is related to XCM. label Apr 18, 2024
@acatangiu acatangiu self-assigned this Apr 18, 2024
@acatangiu acatangiu requested a review from a team April 22, 2024 12:20
@acatangiu acatangiu changed the title Remove XCM SafeCallFilter for chains using Weights::v3 [testnets] remove XCM SafeCallFilter for chains using Weights::v3 Apr 22, 2024
@bkchr bkchr added this pull request to the merge queue Apr 22, 2024
Merged via the queue into paritytech:master with commit 84c294c Apr 22, 2024
139 of 140 checks passed
@acatangiu acatangiu changed the title [testnets] remove XCM SafeCallFilter for chains using Weights::v3 [testnets] remove XCM SafeCallFilter for chains using Weights::v2 Apr 23, 2024
@acatangiu acatangiu deleted the testnets-remove-safecallfilter branch June 25, 2024 14:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
T6-XCM This PR/Issue is related to XCM.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants